Snaji/create persipubsub - #1
Conversation
radomsak
left a comment
There was a problem hiding this comment.
Hi Selim
I only reviewed the readme file. I'll work on the rest in the afternoon
Adam
73e0c51 to
9b9a710
Compare
9b9a710 to
e41f3cd
Compare
radomsak
left a comment
There was a problem hiding this comment.
Review part 1
-
You need to decide whether publisher sends messages to the subscriber, or maybe it only writes messages to the queue and then subscriber read them. Once you make up your mind you should make sure that all the comments are consistent (some of my suggestions might not apply then)
-
Please explain what exactly happens in get_queue_data function offline. Maybe there is a way to not have this function
radomsak
left a comment
There was a problem hiding this comment.
Hi Selim,
The code looks really good! There are only minor comments and changes. You did great work.
I reviewed up to tests. More comes tomorrow afternoon
Two comments for now
- You need to decide whether publisher sends messages to the subscriber, or maybe it only writes messages to the queue and then subscriber read them. Once you make up your mind you should make sure that all the comments are consistent (some of my suggestions might not apply then)
- Please explain what exactly happens in get_queue_data function offline. Maybe there is a way to not have this function
Adam
radomsak
left a comment
There was a problem hiding this comment.
Hi Selim,
I looked at the unit tests. they are very nicely structured and clearly written. I think there is no need to do any big changes. Please see minor 2 comments below
Adam
No description provided.